Skip to content

chore(hooks): add Claude Code hook guard, scrub legacy hooksPath refs - #312

Merged
wphillipmoore merged 2 commits into
developfrom
feature/311-hook-guard
May 26, 2026
Merged

chore(hooks): add Claude Code hook guard, scrub legacy hooksPath refs#312
wphillipmoore merged 2 commits into
developfrom
feature/311-hook-guard

Conversation

@wphillipmoore

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

  • Add PreToolUse hook guard, update settings.json, scrub legacy core.hooksPath references from docs

Issue Linkage

Notes

Add .claude/hooks/guard.sh PreToolUse hook, update settings.json with hook wiring and permissions, update docs.
@wphillipmoore
wphillipmoore merged commit 23bd93b into develop May 26, 2026
19 checks passed
@wphillipmoore
wphillipmoore deleted the feature/311-hook-guard branch May 26, 2026 19:33
wphillipmoore added a commit that referenced this pull request Jun 2, 2026
…ons (#314)

* chore(tooling): refresh managed config to current Vergil tooling/actions

Items 1/3/6/7 of the fleet refresh (epic mq-rest-admin-project/.github#14):
marketplace -> vergil-claude-plugin; embed canonical CLAUDE.md template
(+ vrg-docker-run -> vrg-container-run); ignore .vergil/; cd.yml release
uses secrets: inherit (fixes CD startup_failure). Hook guard (item 2)
already landed via #312. audit COMPLIANT; vrg-validate passes. Refs #313.

* fix(security): invoke bash by absolute path in integration test

CodeQL java/relative-path-command flagged ProcessBuilder("bash", ...)
in MqRestSessionIT (relative command name is PATH-hijackable). Use the
absolute /bin/bash. Refs #313.

* chore(ci): re-trigger CI after transient maven-central resolution flake

Empty commit to re-run CI. A prior run failed in checkstyle:check with a
transient 'Failed to read artifact descriptor for
maven-reporting-api:jar:4.0.0' (Maven Central fetch hiccup), unrelated to
the changes here. Refs #313.

---------

Co-authored-by: Phillip Moore <w.phillip.moore@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant